Skip to content

chore: update cockroachdb-skills submodule (12 new commits)#14

Open
github-actions[bot] wants to merge 1 commit into
mainfrom
chore/update-cockroachdb-skills
Open

chore: update cockroachdb-skills submodule (12 new commits)#14
github-actions[bot] wants to merge 1 commit into
mainfrom
chore/update-cockroachdb-skills

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Updates cockroachdb-skills submodule from a3bd37d3aec14f63182bfaebba91a948654d2cbb to 8716a6593b7ef577e17d87c4d196ba18d9349c52.

Upstream changes

8716a65 address review feedback on prefix rename
ccb1f31 Prefix all skill domain folders with 'cockroachdb-' and add CLAUDE.md
40f823b skills/molt-replicator: align preflight flags and slot/publication defaults with binary
c6d437d skills/molt-fetch + molt-verify: align with current molt binary
b9944c7 skills/setting-up-local-cluster: fix tilde expansion and replica check
a873ba6 skills/designing-multi-region-applications: state the multi-region cluster prereq
6320183 skills/security: drop nonexistent SQL settings and fix invalid GRANT/ALTER syntax
1bb4521 skills: fix broken SQL syntax in privilege checks and TTL example
44fff22 skills: correct outdated zone-vs-cluster-setting facts
4b40099 skills/profiling-{statement,transaction}-fingerprints: correct retention and sampling claims
539c681 skills/analyzing-schema-change-storage-risk: prefer SHOW JOBS over crdb_internal.jobs
970388e skills: replace restricted crdb_internal queries with cockroach node status

This PR was auto-generated by the update-skills workflow.

@viragtripathi

Copy link
Copy Markdown
Contributor

Closing as stale.

This auto-generated PR targets cockroachdb-skills 8716a659 (the "prefix all skill domain folders with cockroachdb- and add CLAUDE.md" change). Upstream main has since been rebased onto a different lineage that dropped that rename — 8716a659 is no longer an ancestor of the current submodule HEAD (84bc1e4), which I verified with git merge-base --is-ancestor.

cursor-plugin is already updated to that current HEAD (84bc1e4), so merging this would revert the skills to an abandoned lineage. The update-skills workflow will open a fresh PR automatically if upstream advances again.

@viragtripathi

Copy link
Copy Markdown
Contributor

Reopening. I closed this earlier on an incorrect premise, and I want to be clear about why.

My earlier analysis was based on a stale local checkout where the skills submodule still pointed at 84bc1e4, and from that I concluded 8716a659 (the cockroachdb- folder prefix rename) had been abandoned upstream. After fetching, that is not the case: claude-plugin main has already adopted 8716a659 (merged via cockroachdb/claude-plugin#18 and #19), so the prefixed layout is the current direction, not an abandoned one.

This PR is the matching update for cursor-plugin. Its base (a3bd37d3) is the current cursor-plugin main, and it advances the submodule to 8716a659, bringing cursor-plugin into parity with claude-plugin. Apologies for the incorrect close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant